Skip to content

chore: Fix the archived server links for modelcontextprotocol/servers #197

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 1, 2025
Merged

chore: Fix the archived server links for modelcontextprotocol/servers #197

merged 1 commit into from
Jul 1, 2025

Conversation

@HareeshBahuleyan
Copy link
Contributor

@HareeshBahuleyan HareeshBahuleyan commented Jul 1, 2025 •
edited by qodo-merge-pro bot
Loading

User description

Some official modelcontextprotocal servers have been moved to servers-archived

And the links have to be updated in their respective schema JSONs.
Reference PRs: https://github.com/modelcontextprotocol/servers/pull/1995/files

PR Type

Other

Description

  • Update homepage URLs for archived MCP servers

  • Change GitHub links from /servers/ to /servers-archived/

  • Fix broken documentation references for 11 servers

Changes diagram

flowchart LR
  A["MCP Registry Files"] -- "Update URLs" --> B["servers-archived Repository"]
  B -- "Fix broken links" --> C["Updated Homepage References"]
Loading

Changes walkthrough 📝

Relevant files
Configuration changes
11 files
aws-kb-retrieval.json
Update homepage URL to archived repository                             
+1/-1     
brave-search.json
Update homepage URL to archived repository                             
+1/-1     
everart.json
Update homepage URL to archived repository                             
+1/-1     
gitlab.json
Update homepage URL to archived repository                             
+1/-1     
google-drive.json
Update homepage URL to archived repository                             
+1/-1     
google-maps.json
Update homepage URL to archived repository                             
+1/-1     
postgresql.json
Update homepage URL to archived repository                             
+1/-1     
puppeteer.json
Update homepage URL to archived repository                             
+1/-1     
sentry.json
Update homepage URL to archived repository                             
+1/-1     
slack.json
Update homepage URL to archived repository                             
+1/-1     
sqlite.json
Update homepage URL to archived repository                             
+1/-1     
Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
    •

    Copilot AI review requested due to automatic review settings July 1, 2025 09:27
    Copilot AI reviewed Jul 1, 2025
    View reviewed changes
    Copy link
    Contributor

    Copilot AI left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    Pull Request Overview

    This PR updates the homepage links in several server schema JSON files to point to the new archived servers repository. The changes replace the previous URLs (using "blob/main") with new ones that use "tree/main" in the archived repository.

    Reviewed Changes

    Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

    Show a summary per file
    File Description
    mcp-registry/servers/sqlite.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/slack.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/sentry.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/puppeteer.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/postgresql.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/google-maps.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/google-drive.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/gitlab.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/everart.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/brave-search.json Updated homepage URL to the archived servers repository
    mcp-registry/servers/aws-kb-retrieval.json Updated homepage URL to the archived servers repository

    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jul 1, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Inconsistent Repository

    The repository URL still points to the main servers repository while the homepage now points to servers-archived. This inconsistency could confuse users about where the actual code resides.

     
      "url": "https://github.com/modelcontextprotocol/servers"
},
"homepage": "https://github.com/modelcontextprotocol/servers-archived/tree/main/src/aws-kb-retrieval-server",

    @qodo-merge-pro
    Copy link
    Contributor

    qodo-merge-pro bot commented Jul 1, 2025

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    General
    Update repository URL consistency

    The repository URL should be updated to match the archived repository since the
    homepage now points to the archived version. This ensures consistency between
    the repository and homepage URLs.

    mcp-registry/servers/aws-kb-retrieval.json [5-8]

     "repository": {
   "type": "git",
-  "url": "https://github.com/modelcontextprotocol/servers"
+  "url": "https://github.com/modelcontextprotocol/servers-archived"
 },

    [To ensure code accuracy, apply this suggestion manually]

    Suggestion importance[1-10]: 8

    __

    Why: The suggestion correctly identifies that since the homepage URL now points to an archived repository, the repository.url should also be updated for consistency.

    		Medium
    • More

    GabrielDrapor
    GabrielDrapor approved these changes Jul 1, 2025
    View reviewed changes
    Copy link
    Contributor

    @GabrielDrapor GabrielDrapor left a comment

    Choose a reason for hiding this comment

    The reason will be displayed to describe this comment to others. Learn more.

    @HareeshBahuleyan Thanks, I missed this before

    @GabrielDrapor GabrielDrapor merged commit 380a077 into pathintegral-institute:main Jul 1, 2025
    3 of 4 checks passed
    @mcpm-semantic-release
    Copy link

    mcpm-semantic-release bot commented Jul 7, 2025

    🎉 This PR is included in version 2.0.0 🎉

    The release is available on GitHub release

    Your semantic-release bot 📦🚀

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Reviewers

    Copilot code review Copilot Copilot left review comments

    @GabrielDrapor GabrielDrapor GabrielDrapor approved these changes

    Assignees

    No one assigned

    Labels

    Review effort 1/5

    Projects

    None yet

    Milestone

    No milestone

    Development

    Successfully merging this pull request may close these issues.

    2 participants

    @HareeshBahuleyan @GabrielDrapor